Skip to content

Wire the contribution leaderboard, city switcher and home distance into their pages - #335

Merged
AnayDhawan merged 2 commits into
StudentSuite:mainfrom
propcgamer20-png:feat/wire-remaining-components
Nov 2, 2026
Merged

AnayDhawan merged 2 commits into
StudentSuite:mainfrom
propcgamer20-png:feat/wire-remaining-components

Conversation

@propcgamer20-png

Copy link
Copy Markdown
Member

What this changes

Part of #313. Wires three of the standalone components that no page imported yet; each checkbox below is a separate item in that issue.

  • ContributionLeaderboard on /about (src/app/about/page.tsx): ranks added_by across the dataset already loaded on that page, no new data or API.
  • CitySwitcher in the homepage hero (src/components/home/hero.tsx): "Jump to a city" going to /city/<slug>, fed by the getCities(places) the hero already computes.
  • RouteEtaCard in the pin popup: PlacesMap now passes the signed-in visitor's saved home ({ lat, lng }) to MapView, which passes it to ClusteredMarkers and PinPopup. With a home set, the popup shows the straight-line distance and a rough walking time; without one it renders nothing.

Not touched, and why: PlaceReviewBadge and RecentAdditionsFeed need data the schema does not have (ratings, added-on dates); HeatmapToggle needs map rendering first; PlaceDetailDrawer, ComparePlacesSheet, SharePlaceCard and PlacePhotoGallery need a design decision on where they open from; CategoryIcon would change the pin-matching legend dots. #313 therefore stays open.

Proof

Not verified

I did not open the pages in a browser, so the layout of the new switcher in the hero and the extra popup line on a small screen are unchecked; the tests cover rendering and props only.

Type of change

  • Code or fix
  • New public place(s)
  • New or updated benefit guide
  • New or updated resource link
  • Docs

@vercel

vercel Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

@propcgamer20-png is attempting to deploy a commit to the studentsuite Team on Vercel.

A member of the Team first needs to authorize it.

@AnayDhawan
AnayDhawan merged commit ff252b9 into StudentSuite:main Nov 2, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants